Skip to content

Fix workspace_swap doc drift in fabric-development-process.md#36

Merged
michaeldeongreen merged 1 commit into
devfrom
docs/workspace-swap-drift-fix
May 9, 2026
Merged

Fix workspace_swap doc drift in fabric-development-process.md#36
michaeldeongreen merged 1 commit into
devfrom
docs/workspace-swap-drift-fix

Conversation

@michaeldeongreen

Copy link
Copy Markdown
Owner

PR #34 hardened workspace_swap.py (mandatory .env, removed interactive prompt fallback, removed value-set-priority shortcut, added YES confirmation, added self-heal recovery pass) and PR #35 moved that confirmation into the chat UI for slash-command invocations. The developer process doc still described the old behavior in three places. This PR brings the doc back in sync.

Changes (1 file, ~10 lines)

  • Step 4 (swap-to-feature) \u2014 drops the 'or prompts if .env is missing' claim; adds the YES confirmation step; mentions the recovery pass.
  • Local .env Setup section \u2014 replaces 'falls back to interactive prompt' + 'value set is source of truth' with the new model (.env always authoritative; value set used by swap-to-dev and the recovery pass; YES confirmation gate).
  • Copilot Chat section \u2014 notes that /swap-to-feature moves the YES confirmation into the chat UI.

Why now

This branch is being PRed alongside the cluster of swap-related changes (PR #34, PR #35) on the next dev \u2192 test \u2192 main promotion. Better to ship the doc with the matching behavior than to leave a known-wrong doc on main.

Tests

185 unit tests still pass (no script changes).

… pass

PR #34 made .env mandatory for swap-to-feature, removed the interactive
prompt fallback, removed the value-set-priority shortcut, and added a
case-sensitive YES confirmation. PR #35 moved that confirmation into the
chat UI for slash-command invocations. fabric-development-process.md still
described the old behavior in three spots.

Updates:
- Step 4 (swap-to-feature): drop the 'or prompts if .env is missing'
  claim; add the YES confirmation step; add the recovery pass step.
- Local .env Setup section: replace 'falls back to interactive prompt'
  + 'value set is source of truth' with the new model (.env always
  authoritative for swap-to-feature; value set used by swap-to-dev
  and the recovery pass; YES confirmation gate).
- Copilot Chat section: note that /swap-to-feature moves the YES
  confirmation into the chat UI.

Tests: 185 pass (no script changes).
@michaeldeongreen michaeldeongreen merged commit c4ee255 into dev May 9, 2026
2 checks passed
@michaeldeongreen michaeldeongreen deleted the docs/workspace-swap-drift-fix branch May 9, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant